From: tsteven4 <tsteven4@gmail.com>
Date: Sun, 3 Mar 2019 17:29:24 +0000 (-0700)
Subject: clang-tidy readability-implicit-bool-conversion on gui.
X-Git-Tag: archive/raspbian/1.10.0+ds-2+rpi1~1^2~12^2~8^2~32^2~7
X-Git-Url: https://dgit.raspbian.org/%22http:/www.example.com/cgi/%22https://%22%22/%22http:/www.example.com/cgi/%22https:/%22%22?a=commitdiff_plain;h=83af77aea9e1d74366bae132caa51040a677c8ef;p=gpsbabel.git

clang-tidy readability-implicit-bool-conversion on gui.
---

diff --git a/gui/filterdata.cc b/gui/filterdata.cc
index fb96fc5fb..5652c8fea 100644
--- a/gui/filterdata.cc
+++ b/gui/filterdata.cc
@@ -35,7 +35,7 @@ QStringList WayPtsFilterData::makeOptionString()
     args << QString("radius,distance=%1%2,lat=%3,lon=%4")
          .arg(radiusVal).arg("MK"[radiusUnit]).arg(latVal, 0, 'f', 8).arg(longVal, 0, 'f', 8);
   }
-  if (duplicates && (shortNames ^ locations)) {
+  if (duplicates && ((shortNames ^ locations) != 0)) {
     args << QString("-x");
     QString s = "duplicate";
     if (shortNames) {
@@ -142,7 +142,7 @@ QStringList TrackFilterData::makeOptionString()
     s += QString(",title=%1").arg(titleString);
   }
 
-  if (s.length()) {
+  if (s.length() != 0) {
     args << "-x" << "track" + s;
   }
 
diff --git a/gui/filterdlg.cc b/gui/filterdlg.cc
index 605eb8347..ca20c3ff2 100644
--- a/gui/filterdlg.cc
+++ b/gui/filterdlg.cc
@@ -125,7 +125,7 @@ void FilterDialog::helpX()
 //------------------------------------------------------------------------
 void FilterDialog::runDialog()
 {
-  if (exec()) {
+  if (exec() != 0) {
     for (int i=0; i<pages_.size(); i++) {
       pages_[i]->getWidgetValues();
       *(usePages_[i]) = ui_.filterList->item(i)->checkState() == Qt::Checked;
diff --git a/gui/mainwindow.cc b/gui/mainwindow.cc
index 3ff7fa4af..26490c7ac 100644
--- a/gui/mainwindow.cc
+++ b/gui/mainwindow.cc
@@ -120,7 +120,7 @@ static QString MakeOptions(const QList<FormatOption>& options)
 static QString MakeOptionsNoLeadingComma(const QList<FormatOption>& options)
 {
   QString str = MakeOptions(options);
-  return (str.length()) ? str.mid(1) : str;
+  return (str.length()) != 0 ? str.mid(1) : str;
 
 }
 
diff --git a/gui/map.cc b/gui/map.cc
index ea9c84ec2..818c5ca30 100644
--- a/gui/map.cc
+++ b/gui/map.cc
@@ -301,7 +301,7 @@ void Map::markerClicked(int t, int i)
 void Map::logTime(const QString& s)
 {
   //  fprintf(stderr, "Log: %s:  %d ms\n", s.toStdString().c_str(), stopWatch.elapsed());
-  if (textEdit_) {
+  if (textEdit_ != nullptr) {
     textEdit_->appendPlainText(QString("%1: %2 ms").arg(s).arg(stopWatch_.elapsed()));
   }
   stopWatch_.start();
diff --git a/gui/optionsdlg.cc b/gui/optionsdlg.cc
index 546d54434..10df747cc 100644
--- a/gui/optionsdlg.cc
+++ b/gui/optionsdlg.cc
@@ -222,7 +222,7 @@ void OptionsDlg::acceptClicked()
 {
   for (int k=0; k<options_.size(); k++) {
     options_[k].setSelected(checkBoxes_[k]->isChecked());
-    if (fields_[k]) {
+    if (fields_[k] != nullptr) {
       if (options_[k].getType() == FormatOption::OPTboundedInt) {
         int value = static_cast<QSpinBox*>(fields_[k])->value();
         value = qMax(qMin(value, options_[k].getMaxValue().toInt()),options_[k].getMinValue().toInt());
diff --git a/gui/processwait.cc b/gui/processwait.cc
index 3dc8a6509..19b027088 100644
--- a/gui/processwait.cc
+++ b/gui/processwait.cc
@@ -186,7 +186,7 @@ void ProcessWaitDialog::finishedX(int exitCode, QProcess::ExitStatus es)
 void ProcessWaitDialog::appendToText(const char* ptr)
 {
   outputString_ += QString(ptr);
-  for (const char* cptr = ptr; *cptr; cptr++) {
+  for (const char* cptr = ptr; *cptr != 0; cptr++) {
     if (*cptr == '\r') {
       continue;
     }
diff --git a/gui/serial_unix.cc b/gui/serial_unix.cc
index 1fb08db30..da8c39a0c 100644
--- a/gui/serial_unix.cc
+++ b/gui/serial_unix.cc
@@ -98,7 +98,7 @@ void MainWindow::osLoadDeviceNameCombos(QComboBox* box)
   const QStringList devices = dynamicDevices();
   box->addItems(devices);
 
-  for (int i=0; deviceNames[i]; i++) {
+  for (int i=0; deviceNames[i] != nullptr; i++) {
     if (!devices.contains(deviceNames[i])) {
       box->addItem(deviceNames[i]);
     }
diff --git a/gui/upgrade.cc b/gui/upgrade.cc
index 73909c85f..5dced1284 100644
--- a/gui/upgrade.cc
+++ b/gui/upgrade.cc
@@ -60,11 +60,11 @@ UpgradeCheck::UpgradeCheck(QWidget* parent, QList<Format>& formatList,
 
 UpgradeCheck::~UpgradeCheck()
 {
-  if (replyId_) {
+  if (replyId_ != nullptr) {
     replyId_->abort();
     replyId_ = nullptr;
   }
-  if (manager_) {
+  if (manager_ != nullptr) {
     delete manager_;
     manager_ = nullptr;
   }
@@ -123,7 +123,7 @@ UpgradeCheck::updateStatus UpgradeCheck::checkForUpgrade(
   args += "&os=" + getOsName();
   args += "&cpu=" + getCpuArchitecture();
   args += "&os_ver=" + getOsVersion();
-  args += QString("&beta_ok=%1").arg(allowBeta);
+  args += QString("&beta_ok=%1").arg(static_cast<int>(allowBeta));
   args += "&lang=" + QLocale::languageToString(locale.language());
   args += "&last_checkin=" + lastCheckTime.toString(Qt::ISODate);
   args += QString("&ugcb=%1").arg(babelData_.upgradeCallbacks_);
@@ -139,14 +139,14 @@ UpgradeCheck::updateStatus UpgradeCheck::checkForUpgrade(
     int rc = formatList_[i].getReadUseCount();
     int wc = formatList_[i].getWriteUseCount();
     QString formatName = formatList_[i].getName();
-    if (rc) {
+    if (rc != 0) {
       args += QString("&uc%1=rd/%2/%3").arg(j++).arg(formatName).arg(rc);
     }
-    if (wc) {
+    if (wc != 0) {
       args += QString("&uc%1=wr/%2/%3").arg(j++).arg(formatName).arg(wc);
     }
   }
-  if (j && babelData_.reportStatistics_) {
+  if ((j != 0) && babelData_.reportStatistics_) {
     args += QString("&uc=%1").arg(j);
   }
 
@@ -282,7 +282,7 @@ void UpgradeCheck::httpRequestFinished(QNetworkReply* reply)
     }
   }
 
-  if (response.length()) {
+  if (response.length() != 0) {
     QMessageBox information;
     information.setWindowTitle(tr("Upgrade"));